Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added filter criteria related Issue 103 #132

Merged
merged 3 commits into from
Feb 9, 2024
Merged

Added filter criteria related Issue 103 #132

merged 3 commits into from
Feb 9, 2024

Conversation

RubenBG7
Copy link
Collaborator

@RubenBG7 RubenBG7 commented Feb 5, 2024

Updated desing guidelines related filtering discussed in issue 103

What type of PR is this?

Add one of the following kinds:

  • documentation

What this PR does / why we need it:

Upgrade Design guidelines

Which issue(s) this PR fixes:

Fixes #103

Updated desing guidelines related filtering discussed in issue 103
patrice-conil
patrice-conil previously approved these changes Feb 5, 2024
Copy link
Collaborator

@patrice-conil patrice-conil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/LGTM

jordonezlucena
jordonezlucena previously approved these changes Feb 6, 2024
Copy link
Collaborator

@jordonezlucena jordonezlucena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/LGTM

@gmuratk
Copy link
Collaborator

gmuratk commented Feb 6, 2024

Dates in the examples seem to be yyyy-dd-mm, which is deviating from the full-date pointed out by @rartych in #93 (comment)

Does it make sense to update the examples as part of this PR?

@rartych rartych dismissed stale reviews from jordonezlucena and patrice-conil via 6975f4e February 6, 2024 15:32
@rartych
Copy link
Collaborator

rartych commented Feb 6, 2024

@gmuratk Thank you for spotting this. I have corrected the dates in the table to be compatible with date-time format.

@rartych rartych requested a review from patrice-conil February 6, 2024 15:37
rartych
rartych previously approved these changes Feb 6, 2024
patrice-conil
patrice-conil previously approved these changes Feb 8, 2024
Copy link
Collaborator

@patrice-conil patrice-conil left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rartych rartych dismissed stale reviews from patrice-conil and themself via 3f59a99 February 9, 2024 11:48
@rartych
Copy link
Collaborator

rartych commented Feb 9, 2024

Conflicts resolved with changes introduced in #123

@rartych rartych self-requested a review February 9, 2024 11:53
@rartych rartych merged commit a830e22 into main Feb 9, 2024
@rartych rartych deleted the RubenBG7-patch-2 branch June 11, 2024 09:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants